Skip to content

Fix build instructions to match actual consumer pattern#211

Merged
jappeace merged 1 commit intojappeace:masterfrom
jappeace-sloth:fix-readme-build-instructions
Apr 20, 2026
Merged

Fix build instructions to match actual consumer pattern#211
jappeace merged 1 commit intojappeace:masterfrom
jappeace-sloth:fix-readme-build-instructions

Conversation

@jappeace-sloth
Copy link
Copy Markdown

Summary

  • Removed incorrect --arg mainModule from nix/apk.nix examples (apk.nix doesn't accept that parameter)
  • Rewrote consumer projects section to show the real pattern: create your own nix files that import nix/lib.nix and call mkAndroidLib/mkApk/mkIOSLib/mkSimulatorApp
  • Fixed iOS setup instructions: replaced manual cp + xcodegen with mkSimulatorApp-based flow
  • Added install.sh, install-wear.sh, and setup-ios.sh script examples matching prrrrrrrrr's working scripts

Test plan

  • README-only change, no code affected
  • Verified all nix function signatures match lib.nix, cross-deps.nix, ios-deps.nix

🤖 Generated with Claude Code

The README had incorrect build instructions:
- apk.nix was shown with --arg mainModule, but it doesn't accept that parameter
- Consumer project section showed importing apk.nix directly with mainModule/consumerCabalFile,
  but the real pattern is creating your own nix files that import lib.nix
- iOS setup showed manual cp + xcodegen, but the real flow uses mkSimulatorApp

Rewrote the consumer sections to match prrrrrrrrr's working build scripts:
- Android: show nix/android.nix with mkAndroidLib + cross-deps.nix, nix/apk.nix with mkApk
- Include install.sh and install-wear.sh script examples
- iOS: show nix/ios.nix with mkIOSLib + ios-deps.nix, nix/ios-app.nix with mkSimulatorApp
- Include setup-ios.sh script pattern with xcodegen

Prompt: fix the build instructions for hatter in the readme, look at prrrrrrrrrr how it's build via those scripts, those are correct, use that as a reference to repair the build instructions

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jappeace jappeace merged commit 6ce2505 into jappeace:master Apr 20, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants